-
Notifications
You must be signed in to change notification settings - Fork 185
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Don't show a source if diagnostic doesn't have a source #2119
Conversation
Needs some more work to address various places that show that info. For example the diagnostics quick panel shows some stray Hover info also missing space in that case: For reference, diagnostics panel looks like this without |
I would probably go with a much simpler solution that uses server name ( |
I'd like to get this behaviour right because there is no need to show a source and getting this behaviour to be closer to VS Code would be better for overall "LSP compliance" I suppose. Gonna dump some comparisons between LSP 1.20 and VS Code here to highlight the differences. I'll try my best to patch this into the LSP plugin. |
Then maybe LSP should follow VSCode here and use parenthesis for |
This comment was marked as resolved.
This comment was marked as resolved.
You should be able to adjust the syntax file to address that. |
but the source/code part does appear to be slightly darker? So I didn't introduce this issue. |
I don't know what this means. For context, before I patched this plugin, I had no experience with Python or the Sublime API, and I'm still just verifying my changes by opening the .sublime-package in 7zip, replacing the files, and restarting ST. |
Actually it's a phantom so it's even easier to change. I'll make a comment in relevant place. |
Co-authored-by: Rafał Chłodnicki <rchl2k@gmail.com>
Co-authored-by: Rafał Chłodnicki <rchl2k@gmail.com>
|
You can ignore that part for now. We'll maybe tweak it later. |
Looks better & it's the same as what VS Code does.